-
Notifications
You must be signed in to change notification settings - Fork 14
add notebook with basic microwave simulation using the impedance calculator #112
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Feel free to take a look @weiliangjin2021 and @QimingFlex. I just need to make some final touches, but the content of the notebook is ready for review. |
|
Ok just missing a rendering. Also not sure how we want to organize this in the docs. I called it a feature tutorial under the "microwave" folder, since it uses a lot of functionality from the microwave plugin. |
tomflexcompute
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very good tutorial. Thanks @dmarek-flex ! For feature tutorials we typically don't need to have a 3D rendering. A few minor comments:
-
The time-domain field in the last plot is very nice. It might be interesting to also plot a frequency-domain reflection spectrum to see the reflection is minimized at the central frequency when the load is matched?
-
"Setup structures and simulation..." -> "Set up structures...". Same for "we need to setup the Tidy3D simulation..." and "Now, the simulation is setup...".
-
In microwave.rst, maybe make the title something like "Microwave and RF" for better SEO?
|
Quite enjoyed going through the tutorial, thanks! A few comments below:
|
added a diagram of the path integrals and finalized flow of cells
2f038f3 to
4a6576d
Compare
|
I finished making the changes you both requested.
I experimented with this, but the plots were not very illuminating because the incident field is also recorded by the frequency domain monitor. I'll leave that type of analysis for the tutorial on the smatrix plugin.
I added a brief discussion about the accuracy. I also did a lot of tests to see what I could do to increase the accuracy of Tidy3D. The most effective changes was increasing the grid resolution. The final accuracy is on the order of 1% which I think is reasonable. |
|
Hello! I also really enjoyed going through this notebook. I liked the way you connect the theory with the simulations, the figures, and how progressive it feels to get familiar with the features. I like how you go from the axis-to-custom voltage/current integral calculations and relate it back to the theory. In terms of the docs, looks correct to me how you've done it! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dmarek-flex Very well-prepared notebook, and I enjoyed reading it as well! A few comments:
-
It seems PML is quite far from the circuit port in the x-y plane. In such a case, the port will be considered open. I think PML should be directly connected to the edge of the microstrip to absorb waves.
-
The diagram is beautiful! But it seems there are some differences between the parameters shown in the schematic diagram and the actual parameters. For example, the thickness of the substrate is 2*height in the simulation but just height in the diagram.
-
Would it be better to plot voltage and current in real units like V and A? This way, users can easily see the impedance by comparing the peaks.
The problem there is then the excitation would be inside the PML. I could have also added a matched load at the incident side, but I wanted to keep it as simple as possible.
The thickness is indeed 2x the height, but the box is centered at z = 0, so the effective height in the simulation is correct. I did that to avoid warnings in Tidy3D when objects are too close to simulation boundaries.
The reason I normalized the voltages and currents is because the absolute values I get are not pleasant. If I remember correctly the voltage is in the tens of thousands of volts, so I think that might be confusing! We would need to change how the UniformCurrentSource is normalized in order to get a more reasonable voltage/current. Incidentally, the default use of microns instead of the |
Can we move the source a bit inward towards the strip so it won't touch the PML? Reflections will be generated at the port, which might affect the accuracy of the characteristic impedance computation. Everything else looks good! |
On your first point, it is not really possible because the source is not directional. If I placed the source somewhere else besides the end of the strip, it will excite fields propagating in both directions. For your second concern, this should not be an issue since in the first simulation where I compute impedance, the PML will not produce any reflections (or at least they are very small). You can see for yourself in the first plots of voltage and current in the time domain, where there are no reflections. |
This is a very introductory style notebook to explain some of the functionality in the microwave plugin.